Wiring up the UI to the events #776

chkeita · 2021-04-06T21:08:09Z

Summary of the Pull Request

The UI now receives the telemetry events
A new section for the coverage has been added
All synced dir are now monitored by the UI
Gracefully exit from the UI

depends on #663

chkeita · 2021-04-06T21:37:53Z

v1

v2

Added new section for the coverage in the UI Added event broadcast in the the telemtry lib Gracefully exit from the UI

src/agent/onefuzz-agent/src/local/tui.rs

bmc-msft · 2021-04-06T23:01:51Z

When I run libfuzzer-fuzz, I don't see inputs as a monitored container.

bmc-msft · 2021-04-06T23:15:09Z

There is a ~30s delay from quitting the agent with q and the atexit code firing.

src/agent/onefuzz-agent/src/local/tui.rs

bmc-msft · 2021-04-06T23:27:22Z

If I run a libfuzzer task with files in the inputs directory, I see all of the files listed show up as a "new_coverage" event. I would not expect these to be reported as new coverage

- swithed to using paragraph to display file count and stats - change color of coverage bar to white

src/agent/onefuzz-agent/src/local/generic_crash_report.rs

ghost · 2021-04-09T17:05:45Z

Hello @bmc-msft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (`@msftbot`) and give me an instruction to get started! Learn more here.

chkeita force-pushed the chkeita/local_run_ui2 branch from 9aabc86 to f535525 Compare April 6, 2021 21:54

wiring all syncdirs to the UI to monitor their file count

9ec4f74

Added new section for the coverage in the UI Added event broadcast in the the telemtry lib Gracefully exit from the UI

chkeita force-pushed the chkeita/local_run_ui2 branch from f535525 to 9ec4f74 Compare April 6, 2021 21:59

chkeita commented Apr 6, 2021

View reviewed changes